Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RFE: add tests for Rust bindings #411

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ManaSugi
Copy link
Contributor

Add the regression tests (1-60) for the libseccomp crate that is Rust language bindings for the libseccomp library.

You can run the tests as follows:

$ sed -i "/^AC_INIT/ s/0.0.0/9.9.9/" configure.ac
$ ./autogen.sh
$ ./configure --prefix=$(pwd)/src/.libs --enable-rust
$ make && make install
$ make check-build
$ cd tests && ./regression -m rust

Based on: #323

@ManaSugi ManaSugi force-pushed the add-rust-tests branch 2 times, most recently from 11ff66d to 6a61aad Compare September 22, 2023 06:32
@ManaSugi ManaSugi changed the title [WIP] tests: Add tests for Rust bindings tests: Add tests for Rust bindings Sep 22, 2023
@ManaSugi
Copy link
Contributor Author

@pcmoore @drakenclimber
All tests passed successfully. But we need to discuss how we control the CI tests with the separate Rust bindings repository as mentioned in #323.

@pcmoore pcmoore changed the title tests: Add tests for Rust bindings RFE: add tests for Rust bindings Nov 30, 2023
@ManaSugi ManaSugi force-pushed the add-rust-tests branch 2 times, most recently from 81decac to aef20cc Compare February 19, 2024 11:12
@coveralls
Copy link

coveralls commented Feb 19, 2024

Coverage Status

coverage: 89.474%. remained the same
when pulling 86f5b73 on ManaSugi:add-rust-tests
into 47ca644 on seccomp:main.

Add the regression tests (1-60) for the libseccomp crate that is
Rust language bindings for the libseccomp library.

You can run the tests as follows:

```sh
$ sed -i "/^AC_INIT/ s/0.0.0/9.9.9/" configure.ac
$ ./autogen.sh
$ ./configure --prefix=$(pwd)/src/.libs --enable-rust
$ make && make install
$ make check-build
$ cd tests && ./regression -m rust
```

Based on: seccomp#323

Signed-off-by: Manabu Sugimoto <Manabu.Sugimoto@sony.com>
Signed-off-by: mayank <mayank.mrinal@sony.com>
@ManaSugi
Copy link
Contributor Author

@pcmoore @drakenclimber
Updated the year of copyright and rebased.
I'd appreciate if you could review this when you have time.

@pcmoore
Copy link
Member

pcmoore commented Feb 20, 2024

Thank you @ManaSugi for your continued efforts here. I wanted to review things with you and @drakenclimber in the main issue tracker: #323 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants